-
-
Notifications
You must be signed in to change notification settings - Fork 636
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sessions UI: visualization, month/year charts #15902
base: master
Are you sure you want to change the base?
Conversation
Das sieht richtig genial aus. Bei kurzem Test ist es mir allerdings nicht gelungen, in diesem Branch die "Charging Sessions" zu öffnen- Menüclick tut gar nichts mehr. Console:
|
Vorsichtshalber die Frage: |
@VolkerK62 was hat das mit Grafiken zu tun? Nix... |
ok, ok. bitte nicht schlagen 😄 |
@VolkerK62 klar, CSV gleibt weiterhin da |
|
Geht weder Safari noch Chrome:
|
Wunderschön ;-) |
Merkwürdig. Ich schau mal ob ich das irgendwie reproduziert bekommen. |
Bin begeistert, super gelungen. 👍🤗 ist echt Hammer geworden |
@StevieC121176 das ist ja noch nicht fertig 😄. Da geht noch was. |
Ich würde sagen, da bleiben keine Wünsche offen. |
Sieht super aus 👍 |
Hi @maatinh, ich kann den Kommentar noch nicht ganz zuordnen. Welche Maßeinheiten meinst du genau? |
Ich glaube er meint in Bild 3 unten die Tabelle. |
Ah ok, aber die Spaltentitel und Maßeinheiten existieren da ja bereits. Der Header (Zeile 1) ist sticky und damit immer sichtbar. Wir wiederholen die Einheiten bewusst nicht pro Zelle. Aber wenn das nicht klar genug ist oder Verbesserungsideen da sind gerne in einem separaten Issue. In diesem PR bleibt die Tabelle der Ladevorgänge eigentlich unverändert. "Nur" die Visualisierung wird ergänzt. |
Sieht klasse aus! Eine Frage: Lässt es sich einrichten, dass die einzelnen Ladevorgänge aus der Tabelle eingeklappt sind, so dass nur die Zeile „Insgesamt“ aus Screenshot 5 sichtbar ist? Schön wäre auch, wenn man die Durchschnittskosten alternativ als €/100km darstellen könnte (bedingt eine Durchschnittsverbrauchsangabe am Fzg). |
Die runden Ecken der Bar Charts scheint es nur teilweise zu geben. Das sieht etwas inkonsistent aus. Sonst grosses Kino 👍🏻 |
Das mit den runde Ecken bei den stacked bar charts ist leider eine Limitierung von Chart.js. Die Abrundung funktioniert nur beim oberen Element des stacks. Ich mach hier schon einiges an Voodoo dass das in den allermeisten Fällen korrekt dargestellt wird. Immer da wo oben noch eine ganz kleine Lademenge ist klappt das aber nicht. Vielleicht macht's Sinn noch nen Schwellenwert zu definieren unter dem wir diese kleinen Mengen dann einfach wegblenden, aber dann verändern wir die Daten (tooltip und so) nur wegen den runden Ecken. Das ist eigentlich doof. |
Würde ich aus genau dem genannten Grund nicht machen. Verbräuche sind ja nie konstant und die metrik macht für andere Geräte wenig Sinn. Wir starten erstmal mit ct/kWh. |
@maatinh wie vorher schon geschrieben. Lass uns in diesem PR nicht über die bereits bestehende Tabellenansicht diskutieren. Das wäre für mich eine separate Geschichte. |
Leider hat aber keine der Ansichten eine Zusammenfassung der Kosten ausser in der Tabelle. Ich fände es gut, wenn auch die Gesamtkosten visualisiert wären. Das zeigt doch den Vorteil von EVs und evcc erst richtig deutlich auf gegenüber Verbrennern. |
@maatinh da bin ich bei dir. Gibt ja auch noch ein Todo hier im PR:
|
This comment was marked as off-topic.
This comment was marked as off-topic.
The problem with this idea: it would only work IF vehicle was only charged at home, and never at work or public chargers. |
Yeah, it depends if you want to show the total costs of the car or only the costs that you can achieve with charging through evcc. |
Welche Bedeutung haben zeitlicher Verlauf von Ladepreis und Emission? Was bedeutet Preis (im Bild 81€) vs. Kosten? |
wip
TODOs